Skip to content

Configure contact email via environment variable#23

Closed
KaushikML wants to merge 1 commit intokeploy:mainfrom
KaushikML:Kaushik_Roy
Closed

Configure contact email via environment variable#23
KaushikML wants to merge 1 commit intokeploy:mainfrom
KaushikML:Kaushik_Roy

Conversation

@KaushikML
Copy link
Copy Markdown

Summary

  • use SITE_OWNERS_EMAIL environment variable in sendEmail.php
  • document SITE_OWNERS_EMAIL configuration in README
  • add PHPUnit with a test for the env variable
  • run tests in CI
  • provide alt text for testimonial images

Testing

  • phpunit --configuration phpunit.xml

Signed-off-by: Achanandhi-M <achanandhi.m@gmail.com>
Signed-off-by: Kaushik Roy <roykaushik354@gmail.com>
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you and congratulations 🎉 for opening your very first pull request in writers-program

@KaushikML KaushikML closed this Jun 18, 2025
@KaushikML KaushikML deleted the Kaushik_Roy branch June 18, 2025 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants